GtkNotebook: Unify two private variables

Button state was being kept in two separate variables, which lead
to slight confusions in DnD that caused the notebook to ignore the
first click after DnD happened from (within) it. Unify these two
into one, which helps us keep better track of the really pressed
buttons.

container: Don't create too-large clips

gdk_rectangle_union will happily add all the worlds pixels
to the union if the initial rectangle is initialized to all
zeros. Therefore, explicitly check for an empty rectangle
before calling it.

Clarify docs for gdk_rectangle_union

This function does not ignore empty rectangles. Since this
is a fairly subtle point about the behavior, it is worth
spelling this out in the documentation. We've had a bug
open about this for a long time:

https://bugzilla.gnome.org/show_bug.cgi?id=464528

wayland: Protect against NULL offers on gdk_drag_status() implementation

If we're called untimely, we might end up crashing here when poking the
NULL wl_data_offer.

gtkdnd: Disconnect GdkDragContext signals on gtk_drag_source_info_free()

Otherwise it may still be tricked into emitting further signals at a
time we've already destroyed the GtkDragSourceInfo passed as callback
user data.

notebook: Get rid of during_reorder variable

Use operation == OPERATION_REORDER instead.

notebook: Properly hide drag window

Move code to properly reinsert the tab label to where it belongs.
The if has the distinction between reparented-to-dnd-window and
just-changed-the-gdk-window-to-draw-to right there.

https://bugzilla.gnome.org/show_bug.cgi?id=760754

notebook: Get rid of during_detach member

use operation == OPERATION_DETACH instead.

Add a function to get affected area for background

The new function, gtk_render_background_get_clip answers the
question: what pixels are affected if I call gtk_render_background ?

The long-term goal is to have APIs that answer this question for
all rendering primitives.

file chooser: Allow saving even if executable attribute is not set

Commit 8e975b2 (Bug 753969) introduced check of parent accessibility.
Consequently it is not possible to save file if executable attribute
is not set, which might happen for some gvfs backends. Let's assume
that the folder is accessible even if the attribute is not set.

https://bugzilla.gnome.org/show_bug.cgi?id=760881

configure: Require wayland >= 1.9.91

Needed since commit 1045dda0358dee67e8dfbf13b40181b4c2ba1f1c

viewport: Don't destroy pixelcache while it's still used

Removing the child unsets the pxielcache's style context...

gtk-demo: Fix font features sources

The resources had the prefix, causing the ui file not
to show up.

viewport: Set the pixel cache background from the child

The viewport itself doesn't move, so we cannot use it as the pixel
cache's background. Use the bottommost using element instead, which is
the viewport's child.

This might need adaptations in themes as we want the backgroud to be
opaque to speed up pixel cache performance.

notebook: Implement rtl flipping for CSS nodes

Use gtk_box_gadget_reverse_children and gtk_css_node_reverse_children
to flip the children of the header_gadget and the tabs_gadget when
appropriate.

Add new CSS node tests to verify that the node order is updated
as expected in all cases.

css: AFFFECTS_TEXT causes a resize, not a redraw

And we schedule the resize conditionally only when text is rendered but
don't do that check for the redraw.

widget: Properly invalidate when CSS font properties change.

Updated POTFILES.in and POTFILES.skip

3.19.7

Update expected output of a11y tests

This reflects my recent change to messagedialog titles.

Simplify foreign drawing example

This is code will be copied, so take out unnecessary complication.

Use actual drop status

We have this information in the XDND protocol, so we don't have
to blindly assume that the drop succeeded.

gdk: Document the functions that don't need calling on managed DnD

These functions will be automatically called by the windowing backend.
The usual hooks to run this from in gtk/ shouldn't even happen, but
it is worth to document which calls are expected and which aren't.

wayland: Set weak reference on the current grab window

If the grab window is destroyed the grab will be implicitly removed,
although we won't get GdkSeat:ungrab called in order to clear our
internal window<->seat relation entirely. Setting a weak ref will
nullify the pointer we keep on the seat to the window, avoiding the
expected crashes.

wayland: Unset button modifiers on pointer enter

Due to implicit grabs, we basically can guarantee that the pointer
won't have any buttons pressed at the time of wl_pointer.enter.
Seems like a good place to unset any button modifiers that might
have been left stale by compositor grabs.

wayland: Implement DnD actions as per wl_data_device v3

Implement as a managed GdkDragContext, which actually fits nicely
with the drag-and-drop model in wayland.

wayland: Add internal gdk_wayland_seat_set_global_cursor()

This can be used for cases (like DnD) where there isn't necessarily
a grab, but we want a global pointer cursor.

gtkdnd: Reset widget controllers when starting DnD.

Otherwise there's no full guarantees that this will happen.

gtk: Add private gtk_widget_reset_controllers()

This is a one-shot call that can be called on the places we
know we're taking input away.

gtkdnd: Optionally use gdk_drag_context_manage_dnd()

When this is in use, there's essentially a bunch of dead code here.
When all backends are ported, we'll be able to remove grab/cursor
management plus a bunch of source-side event handlers.

x11: Implement gdk_drag_context_manage_dnd()

This includes managing input events and source-side DND events,
as well as setting the appropriate cursor and emitting the signals
that are expected in this mode of operation.

gdk: Add gdk_drag_get_cursor()

This function (most similar to gtk_drag_get_cursor() helps figure out
the right cursor that applies to a given action. To be used by the
various backends.

gdk: Run DnD internal handlers before the main event handler

We'll be stealing those to GTK+, if the GdkDragContext manages
the DnD operation.

gdk: Allow internal management of source-side DnD

We've traditionally left GTK+ to handle the input side of things,
letting GDK only manage the windowing-specific messaging. This
way of splitting responsibilities is not compatible however with
some backends, we must fold then input management at the DnD stage
into GDK (and backends) domain.

The gdk_drag_context_manage_dnd() call is meant to be the entry
point for this mode of operation, if the drag and drop operation
becomes managed, the caller (i.e. gtkdnd.c) doesn't need to perform
grabs, nor manage input events itself.

As a consequence of this, different aspects now belong to the
backend GdkDragContext implementation:
- Because the caller doesn't see keyboard events anymore,
  keyboard navigation must be managed in GDK, so is the decision
  of the current action based on modifiers/button pressed.
- Because the caller won't see input events in general, the lifetime
  of the drag and drop operation is now communicated through the
  ::drop-performed, ::dnd-finished and ::cancel events
- Because the caller doesn't participate anymore on the action
  being chosen, the pointer cursor must be set by the backend.
  The caller is rather notified of the final action through the
  ::action signal.

The caller is still responsible of dealing with the corresponding
GdkSelection, ensuring its ownership and communicating the supported
mimetypes.

widget: Call gdk_window_mark_paint_from_clip() again

The proper window to call it is the event window, as the call itself
ignores non-native windows anyway.

Fixed typo childen->children

gtk-demo: Add a progress bar to foreigndrawing

This is another commonly requested widget.

container: Properly reorder no-window children

... that are setup with gtk_widget_set_parent_window().

Fixes scrollbars not being drawn in GtkScrolledWindow.

widget: Add forgotten push_group code

... and remove the also forgotten void function that lingered around
with it.

Fixes opacity=0 parts like inactive spinners or sort indicators in
treeview headers being drawn since last commit.

Oops.

widget: Redo drawing code

Previously, we had a special cae to draw subwindows of widgets.

This is not necessary as conformant widgets should be able to properly
render themselves when all windows need to be painted.
From now on assume that is the case.

We therefore paint nonnative GDK windows "inline" by just returning TRUE
for gtk_cairo_should_draw_window() for those windows.

This speeds up hilighting different rows in the listbox gtk-demo example
tremendously (by a factor of 10 or more) as the previous code was
O(<number of non-window subwidgets> *
<number of subwindows>) which in the listbox example were ~15,000 and
~2,000 respectively.

actionbar: Don't forall() widgets twice

When using forall(), only list the revealer, which lists the box
containing all the children. When using foreach(), bypass revealer and
box and list all children added to the box.

Updates

text view: Expose cursor blink control

Derived classes like GtkSourceView with their own ::key-event
handler need access to this, in order to make their keynav
as nice as the builtin one, wrt to caret visibility.

https://bugzilla.gnome.org/show_bug.cgi?id=760748

x11: Set event->scroll.is_stop

We still figure this out from 0/0 scroll events. This method is
not intended to last forever, but it's something we can cling to
so far.

https://bugzilla.gnome.org/show_bug.cgi?id=756729

wayland: add gdk_event_is_scroll_stop_event()

And use it to handle kinetic scrolling in the GtkScrolledWindow.

However, dropping the delta check causes the X11-based kinetic
scroll to break since we don't have the stop event here. Correct handling of
xf86-input-libinput-based scroll events is still being discussed.

https://bugzilla.gnome.org/show_bug.cgi?id=756729

wayland: add support for wl_pointer frame/axis_source/axis_discrete/axis_stop

This adds support for the new wl_pointer events available in v5.

The wl_pointer.axis_source events can be ignored for the purposes here, the
main reason they exist is so that the combination of axis_source=finger and
axis_stop triggers kinetic scrolling. We don't need to care about the source,
axis_stop is enough for us to tell us when we're scrolling.

The wl_pointer.frame events group events together and is intended as a
mechanism to coalesce events together. This for example allows us to now
send a single GTK scroll event for a diagonal scroll. Previously, the two
wl_pointer.axis events had to be handled separately.

The wl_pointer.axis_discrete event sends mouse wheel clicks where
appropriate, and is translated into up/down/left/right scroll events.

https://bugzilla.gnome.org/show_bug.cgi?id=756729

Drop a reference to regions

Regions don't work anymore, so don't recommend them in the docs.

im context simple: Protect shared data structures

Since a41f02f9b1843e0f0085f801430e55f413a9bf9c, GtkIMContextSimple
uses threads to load X Compose files. It does that every time a new
im context object is initialized, so we can easily end up with multiple
threads accessing the shared global_tables list at the same time.

Use a lock to prevent that.

https://bugzilla.redhat.com/show_bug.cgi?id=1276432

cssnode: Don't leak the style cache on destruction

gdkdisplay-x11: Plug memory leak

text view: Update selection node visibility

Make the node invisible if we don't have a selection.

entry: Expand the css node docs

Mention the context menu.

text view: Expand CSS node docs

Mention the selection node, and the context menu.

Forgotten file

notebook: Don't crash when dnd'ing tabs out

https://bugzilla.gnome.org/show_bug.cgi?id=760754

accellabel: Remove unused variable

gdk: Clarify docs about scale

Explicitly state where we return application pixels as opposed
to device pixels.

Trivial typo fix

Add more CSS node tests for notebooks

entry: Deprecate shadow-type

Things like that should be handled using css. Also, the property value
is unused inside GtkEntry.

W32: Detect ticks wraparound

https://bugzilla.gnome.org/show_bug.cgi?id=748327

Add CSS node tests for checkbutton rtl flipping

Add css node tests for expander rtl flipping

expander: Flip the alignment of the vertical box

To ensure that the title moves to the other side as expected
in RTL, use GTK_ALIGN_START/END instead of GTK_ALIGN_FILL
when packing the title gadget into the vertical box, and
flip the alignment when the text direction changes.

expander: Handle rtl changes again

Now that the box gadget has api for it, use it here to
change the order of the gadgets for right-to-left.

checkbutton: Handle rtl changes again

Now that the box gadget has api for it, use it here to
change the order of the gadgets for right-to-left.

box gadget: Add a function to reverse children

This can be used to handle text direction changes.

reftests: Fix up for easy CSS changes

This is mostly search and replace ala
  GtkButton => button
  GtkWindow => window
  .button => button
or removing style properties that aren't used anymore like
  -GtkButton-default-border: 0

reftests: Remove test

We no longer match subtypes, so remove the test for it.

checkbutton: Use a box gadget

Now that builtin icons have a baseline, we can just use
a box gadget with baseline alignment for the indicator
and the label.

builtin icon: set a baseline

Most uses of builtin icons (check and radio buttons,
expanders, etc) are placed next to labels, so they should
be properly positioned wrt to the baseline. Lacking anything
better, give the builtin icons a baseline that places the
center of the icon at the strikethrough position.

box gadget: Deal with optional arguments

The baseline arguments to the preferred size callback can
be NULL. Don't crash if they are.

Trivial formatting fix

Document restrictions on gtk_style_context_get

This makes it official that passing a different state
is not generally supported in the getters for CSS properties.

notebook: show tabs when asked to

gtk_notebook_set_show_tabs must make the header gadget
visible when show_tabs is TRUE, not the other way around.

https://bugzilla.gnome.org/show_bug.cgi?id=760596

accel label: Fix RTL handling

The 'mad hack' that GtkAccelLabel used to affect the GtkLabel
draw function broke with the introduction of gadgets, since
the positioning is no longer relative to the widgets' allocation
at the time of the call, but rather to the gadgets allocation.

Instead of coming up with an even madder hack to keep this
working, give the GtkLabel draw function knowledge about accel
labels.

https://bugzilla.gnome.org/show_bug.cgi?id=760663

inspector: Make cursor-size combo wider

Cursor sizes typically have 2 digits, lets not clip them.

Expand window style class documentation a bit

Adwaita: Keep entry progress visible

When the entry is focused, make the progress one pixel wider
so it doesn't get swallowed by the focus outline.

stylechange: Do the right thing when old and new style are equal

We don't need to compare all their CSS values to figure out that nothing
changed. We know that.

cssnode: Redo GTK_DEBUG=no-css-cache

Previously this setting would just invalidate the whole CSS tree and
thereby hopefully avoid all cache usage.

Now, we actually don't cause extra invalidations anymore, but instead
avoid ever inserting anything into the cache when this setting is set.

cssnode: Implement the style cache

This essentially copies the previous cache implementation. With one
caveat: It is now attached to and maintained by the CssNode, not by the
CssStyle.

And this is important because styles may be reused in incompatible
situations which would cause cache collisions and lead to broken CSS in
weird situations.

cssnode: Split out the style cache

For now, the split out style cache doesn't cache anything. This is
mostly to make sure that bisections of wrong caching behavior will
bisect down to the commit that actually adds caching.

Adwaita: stop axing toolbar separators

Adwaita: style textview derived widgets

color scale: Improve themability

Add the .color class, so themes can style scale.color.
Clip the trough rendering to the content path, so that
themes can set rounded corners.

treeview: Use foreground color to draw dnd marker

This is still not perfect, but makes the dnd marker better visible
in HighContrastInverse.

frame: Don't overlap label and child

We didn't do this previously, so don't start now.

https://bugzilla.gnome.org/show_bug.cgi?id=760528

Adwaita: combo style cleanup and fixes

fixes https://bugzilla.gnome.org/show_bug.cgi?id=760524

combo: Add .linked class when appropriate

Add the .linked style class to combo boxes which have
an entry, since that is what they are.

combobox: Document the popup window CSS node

application: Protect against double shutdown

It is not something that can happen normally.
But if it does, we shouldn't crash.

https://bugzilla.gnome.org/show_bug.cgi?id=760680

mir: add default GdkSeat for the default GdkDisplay

widget: remove queue_resize in set_sensitive

Calling gtk_widget_propagate_state is enough to queue a resize if
needed

See da7a4089fe82981d1412aaa295e76d47a81ad6df

wayland: Fix an endless loop

Adwaita: remove messagedialog label squashing hack

since Matthias fixed this properly.

message dialog: Prevent empty title from being shown

We don't want to grow the titlebar by showing an empty label.

Adwaita: some dialog love

addresses https://bugzilla.gnome.org/show_bug.cgi?id=760525

expander: Destroy gadgets after child widgets

With child widgets now being sorted into box gadgets, bad things
happen if those gadgets disappear prematurely.

notebook: Destroy gadgets after chaining up

While chaining up, we might still call into the notebook code. And that
code expects the gadgets to be there.

https://bugzilla.gnome.org/show_bug.cgi?id=760640